Skip to content

fix: contain legacy session saves under test_history - #556

Open
m-jay21 wants to merge 1 commit into
GreyDGL:mainfrom
m-jay21:fix/502-session-path-containment
Open

m-jay21 wants to merge 1 commit into
GreyDGL:mainfrom
m-jay21:fix/502-session-path-containment

Conversation

@m-jay21

@m-jay21 m-jay21 commented Sep 17, 2026

Copy link
Copy Markdown

Summary

Fixes #502

Test plan

  • `uv run python -m pytest tests/legacy/test_session_save.py -v`
  • `make ci-quick` (140 root tests + agent checks)

Reject traversal/absolute/nested session names, reprompt on invalid
input, and add regression tests for GreyDGL#502.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Legacy session name can write outside test_history

1 participant